Skip to content

Add integration tests for decrypting Ed25519 keys with AES-256-GCM and ChaCha20-Poly1305#23

Merged
nedithgar merged 1 commit intomainfrom
test/add-tests
Oct 14, 2025
Merged

Add integration tests for decrypting Ed25519 keys with AES-256-GCM and ChaCha20-Poly1305#23
nedithgar merged 1 commit intomainfrom
test/add-tests

Conversation

@nedithgar
Copy link
Copy Markdown
Owner

Introduce integration tests to verify the decryption of Ed25519 keys using AES-256-GCM and ChaCha20-Poly1305 ciphers with ssh-keygen. These tests ensure that the decryption process works correctly and that the output matches the expected public key.

Copilot AI review requested due to automatic review settings October 14, 2025 16:50
@nedithgar nedithgar merged commit 8aba0a4 into main Oct 14, 2025
4 checks passed
@nedithgar nedithgar deleted the test/add-tests branch October 14, 2025 16:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds integration tests to verify that Ed25519 keys encrypted with AES-256-GCM and ChaCha20-Poly1305 ciphers can be properly decrypted by ssh-keygen. These tests ensure compatibility between the SwiftKeyGen library's encryption implementation and the standard ssh-keygen tool.

Key changes:

  • Added two new test methods for Ed25519 key decryption verification
  • Tests both AES-256-GCM and ChaCha20-Poly1305 encryption ciphers

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

#expect(ourNormalized == theirNormalized, "Public keys should match")
}
}

Copy link

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove trailing whitespace at the end of the file to maintain consistent code formatting.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants